Skip to content

Conversation

@stephenfin
Copy link

  • Remove redundant packaging config from setup.cfg
  • Remove redundant pytest config from setup.cfg
  • Move tox configuration to tox.ini
  • Move coverage configuration to pyproject.toml

This is already indicated by pyproject.toml.

Signed-off-by: Stephen Finucane <[email protected]>
Once again, this is already indicated by pyproject.toml.

Signed-off-by: Stephen Finucane <[email protected]>
This might be better placed in pyproject.toml, but I have no idea
how to do that while preserving the factor-based configuration.

Signed-off-by: Stephen Finucane <[email protected]>
@GrahamDumpleton
Copy link
Owner

Not sure how safe it is to completely remove setup.cfg file. Things are confusing due to historical move from setup.py, to setup.py/setup.cfg combination, to pyproject.toml. Not sure there isn't still something dependent on setup.cfg metadata for some versions of Python when installing from pure source code (not a wheel), or even for cibuildwheel itself. This is all going to have be explored more throughly before make changes.

First step is to look at just tox and pytest configuration as first step.

@GrahamDumpleton
Copy link
Owner

Rather than merge this, have made changes against develop branch to achieve same aims, but so can do it in parts, properly test and resolve any issues.

As I mentioned would do above, have extracted out configs into tox.ini and pytest.ini as appropriate. Have also dropped running of coverage as that wasn't being used and was in an unknown working state. Also found some other problems with GitHub actions tox configuration which had to be fixed up.

The general project meta and build config is still in setup.cfg in develop branch this has to go somewhere still and can't just be removed as this patch was doing.

So this PR is being superseded, but will leave open for now as will record progress of changes here.

@GrahamDumpleton
Copy link
Owner

Everything now dealt with in develop branch. If need to test for some reason use 2.1.0dev1 from PyPi.

@stephenfin
Copy link
Author

Sounds good. Thanks @GrahamDumpleton

@stephenfin stephenfin deleted the packaging branch January 15, 2026 07:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants